* (bug 8401) Fix regression in SORBS lookup for some DNS setups

The final '.' on the hostname got removed when it was moved out to a config variable.
If the local domain is a wildcard, this can lead to surprising false positives
when the lookup returns a local address instead of failure.

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 6ae3e2a..8d4745c 100644 (file)
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -420,6 +420,7 @@ it from source control: http://www.mediawiki.org/wiki/Download_from_SVN
   in cache key. Probably this is still a bit wonky, though. Was breaking
   parser tests when Chinese tables were getting loaded into Serbian code.
 * (bug 8380) Be nicer about blank lines in deleteBatch.php
+* (bug 8401) Fix regression in SORBS lookup for some DNS setups
 
 
 == Languages updated ==

diff --git a/includes/DefaultSettings.php b/includes/DefaultSettings.php
index 3da4b44..511bf8f 100644 (file)
--- a/includes/DefaultSettings.php
+++ b/includes/DefaultSettings.php
@@ -2072,7 +2072,7 @@ $wgDisableHardRedirects = false;
  * Use http.dnsbl.sorbs.net to check for open proxies
  */
 $wgEnableSorbs = false;
-$wgSorbsUrl = 'http.dnsbl.sorbs.net';
+$wgSorbsUrl = 'http.dnsbl.sorbs.net.';
 
 /**
  * Proxy whitelist, list of addresses that are assumed to be non-proxy despite what the other